Skip to content

Halyna R.#5

Open
halyna1995 wants to merge 3 commits into
HackYourAssignment:mainfrom
halyna1995:week6/halyna1995
Open

Halyna R.#5
halyna1995 wants to merge 3 commits into
HackYourAssignment:mainfrom
halyna1995:week6/halyna1995

Conversation

@halyna1995

Copy link
Copy Markdown

Complete week 6 Azure assignment

@lassebenni lassebenni closed this Jun 10, 2026
@lassebenni lassebenni reopened this Jun 10, 2026
@github-actions

This comment has been minimized.

Comment thread src/pipeline.py
Comment thread src/pipeline.py
Comment thread README.md

@AgneseG AgneseG left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Core pipeline exists, but misses important assignment requirements: no personal schema/search_path, wrong table name, no sslmode=require, README formatting issue.

For next submissinos, pls write a description of the PR too, thanks!

Added a command to list Azure storage blobs.
@github-actions

This comment has been minimized.

Refactor PostgreSQL writing logic to use SQLAlchemy and Pandas.
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 70 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Week 6 Autograder ===
  ✓ PASS  found Dockerfile
  ✓ PASS  found requirements.txt
  ✓ PASS  found src/pipeline.py
  ✓ PASS  found AI_ASSIST.md
  ✓ PASS  found README.md
  ✓ PASS  found docs/ directory
  ✓ PASS  Level 1: required files (10/10 pts)
  ✓ PASS  requirements.txt pins azure-storage-blob
  ✓ PASS  requirements.txt pins psycopg2-binary
  ✓ PASS  Level 2: pinned dependencies (10/10 pts)
  ✓ PASS  Dockerfile uses a python:3.11 base image
  ✓ PASS  Dockerfile copies requirements before src/ (layer cache stays warm)
  ✓ PASS  Dockerfile has a real CMD instruction
  ✓ PASS  Level 3: Dockerfile (10/10 pts)
  ✓ PASS  pipeline.py reads POSTGRES_URL and AZURE_STORAGE_CONNECTION_STRING from env
  ✗ FAIL  pipeline.py does not use 'from contextlib import closing' + 'with closing(...)' (Chapter 4 pattern)
  ✓ PASS  pipeline.py silences the azure SDK logger
  ⚠ WARN  pipeline.py still contains 'raise NotImplementedError' — finish the stubs before submitting
  ✓ PASS  Level 4: pipeline shape (10/15 pts)
  ✗ FAIL  pipeline.py does not use ON CONFLICT ... DO UPDATE (idempotent upsert)
  ✗ FAIL  pipeline.py does not use %s placeholders for parameterised SQL
  ✓ PASS  Level 5: idempotent upsert (0/15 pts)
  ✓ PASS  connection string includes sslmode=require
  ✓ PASS  pipeline.py uses the azure-storage-blob SDK (BlobServiceClient)
  ✓ PASS  Level 6: connection + SDK (10/10 pts)
  ✗ FAIL  AI_ASSIST.md has section headers but is not filled in (1148 chars, 0 TODO line(s); target 1800+ chars, 0 TODOs)
  ✓ PASS  Level 7: AI report (0/10 pts)
  ✓ PASS  README.md has '## Verification' heading and embeds a docs/ image
  ✓ PASS  Level 8: README verification (10/10 pts)
  ✓ PASS  execution-history screenshot present at execution_history.png (73508 bytes)
  ✓ PASS  Level 9: execution screenshot (10/10 pts)
  ✓ PASS  .gitignore correctly excludes __pycache__/, *.pyc, and .env

Score: 70 / 100  (passing: 60)  pass=true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants